Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add the wrappers for glm #32

Merged
merged 8 commits into from
Aug 19, 2024
Merged

add the wrappers for glm #32

merged 8 commits into from
Aug 19, 2024

Conversation

clarkliming
Copy link
Collaborator

@clarkliming clarkliming commented Aug 13, 2024 •
edited
Loading

close #28
close #29

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 13, 2024 •
edited
Loading

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  --------------------
R/bias.R                        11       0  100.00%
R/predict_couterfactual.R       46       0  100.00%
R/prediction_cf.R                3       0  100.00%
R/robin_glm.R                   27       0  100.00%
R/treatment_effect.R            72       4  94.44%   30, 43, 65, 77
R/utils.R                       21       1  95.24%   16
R/variance_anhecova.R           45      19  57.78%   30, 45-46, 59, 65-80
R/variance_hc.R                  9       0  100.00%
TOTAL                          234      24  89.74%

Diff against main

Filename                Stmts    Miss  Cover
--------------------  -------  ------  --------
R/robin_glm.R             +27       0  +100.00%
R/treatment_effect.R        0      -7  +9.72%
TOTAL                     +27      -7  +4.72%

Results for commit: 86e38a3

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 13, 2024 •
edited
Loading

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
robin_glm 👶 $+0.13$ $+10$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
robin_glm 👶 $+0.05$ h_interaction_works_correctly
robin_glm 👶 $+0.07$ robin_glm_works_correctly
treatment_effect 👶 $+0.01$ treatment_effect_works_as_expected_for_custom_contrast
treatment_effect 👶 $+0.02$ treatment_effect_works_for_lm_glm_object

Results for commit cf829a3

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 13, 2024 •
edited
Loading

Unit Tests Summary

 1 files   6 suites   0s ⏱️
19 tests 13 ✅ 6 💤 0 ❌
55 runs  49 ✅ 6 💤 0 ❌

Results for commit 86e38a3.

♻️ This comment has been updated with latest results.

@clarkliming clarkliming merged commit f9d5c12 into main Aug 19, 2024
25 checks passed
@clarkliming clarkliming deleted the 28_robin_wrapper branch August 19, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

stop if vcovHC is used in models not using difference and includes interaction add glm wrapper
1 participant
@clarkliming